Skip to content

Document MSVC doesn't straddle bit-fields#3376

Merged
colin-home merged 2 commits into
MicrosoftDocs:masterfrom
workingjubilee:patch-2
Sep 20, 2021
Merged

Document MSVC doesn't straddle bit-fields#3376
colin-home merged 2 commits into
MicrosoftDocs:masterfrom
workingjubilee:patch-2

Conversation

@workingjubilee
Copy link
Copy Markdown
Contributor

This PR documents existing behavior in the MSVC toolset.

@PRMerger15
Copy link
Copy Markdown
Contributor

@workingjubilee : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver
Copy link
Copy Markdown
Contributor

#label:"aq-pr-triaged"

@PRMerger7 PRMerger7 added the aq-pr-triaged Tracking label for the PR review team label Sep 20, 2021
@PRMerger13
Copy link
Copy Markdown
Contributor

@corob-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Copy link
Copy Markdown
Contributor

@colin-home colin-home left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@workingjubilee
Thanks for the contribution. We mention this implementation-specific behavior in a very oblique way in Padding and alignment of structure members, and in the Bitfields section of x64 software conventions (though the compiler behavior isn't x64-specific). Those pages aren't necessarily where people would go to find this out, so this seems a good place to add that info. I've modified your content to follow our voice and internationalization guidelines.

@colin-home colin-home merged commit 8c25f7f into MicrosoftDocs:master Sep 20, 2021
@workingjubilee
Copy link
Copy Markdown
Contributor Author

Ah, I missed that. Thank you, cross-referencing it does seem appropriate.

@workingjubilee workingjubilee deleted the patch-2 branch September 21, 2021 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants